Skip to content

Conversation

@zeyi2
Copy link
Member

@zeyi2 zeyi2 commented Nov 8, 2025

No description provided.

@zeyi2 zeyi2 changed the title [clang-tidy][NFC] Add clang-tidy formatting commits to .git-blame-ignore-revs [clang-tidy][NFC] Add clang-tidy formatting commit to .git-blame-ignore-revs Nov 8, 2025
@zeyi2
Copy link
Member Author

zeyi2 commented Nov 11, 2025

silent ping :)

@zeyi2 zeyi2 requested a review from vbvictor November 18, 2025 09:56
#
# $ git config blame.ignoreRevsFile .git-blame-ignore-revs

# [clang-tidy][NFC] Enforce 80 characters limit
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is a remaining commit to be added: #168049

I'll add it when the PR above is merged :)

#
# $ git config blame.ignoreRevsFile .git-blame-ignore-revs

# [clang-tidy][NFC] Enforce 80 characters limit
Copy link
Contributor

@vbvictor vbvictor Nov 18, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we should add this at the end of the file to make it chronological order.

@zeyi2 zeyi2 requested a review from vbvictor November 20, 2025 02:45
Comment on lines 147 to 150
# [clang-tidy][NFC] Fix alphabetical order in list.rst and ReleaseNotes.rst
80a65c36625900e37f35c913977e351018ec6d87
0013b5f83d8e3310990d18b8ce581aa2831ea5c5
fddfc705d03ec6e9c75441d66057c1219902c8c1
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think these are worth putting here.
ignore-revs files should be used for large-scale refactoring and where changes may hurt readability.
These changes don't hurt IMO

zeyi2 and others added 2 commits November 20, 2025 20:56
Co-authored-by: Baranov Victor <bar.victor.2002@gmail.com>
@zeyi2 zeyi2 requested a review from vbvictor November 20, 2025 13:07
@zeyi2 zeyi2 merged commit 150d9b7 into llvm:main Nov 20, 2025
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants